wayland: Make display closing work

The code in _gdk_wayland_window_dispose was not safe against
being called twice - it would call g_hash_table_destroy twice
on the known_globals hash table, the second time operating on
freed memory. It was also leaking the list of async_roundtrips.

After fixing both of these issues, the displayclose testcase
now works on Wayland.

wayland: Be more careful when destroying windows

We call gdk_wayland_window_hide_surface when the window gets
destroyed, and in this case, the frame clock might not exist
anymore.

This was showing up in the displayclose testcase.

Trivial formatting fix

wayland: Don't assert non-recursing in gdk_window_destroy

While we do not have subwindows in Wayland, we do create an
artificial root window. When the display is closed, the root
window gets destroyed, causing recursing to be true for the
toplevel windows.

Trivial formatting fix

Updated Greek translation

Updated Greek translation

Adwaita: remove .list-row.activatable reset.

Since being 'activatable' istead of 'button' now that reset
is not needed anymore, the patch is pretty noisy since sass
interpreter changes, those look innocuous though.

Work towards succeeding make check under Wayland

Skip tests that don't work under Wayland, such as tests for GtkPlug
and GtkSocket.

plug/socket: Warn if used outside X11

Give a clear warning when either of these are realized on a non-X11
screen.

Make display test not fail under Wayland

This test really only works under X, since it relies on modifying
the DISPLAY env var.

places view: Don't spew needlessly

The check here was obviously meant to filter out cancellations,
not warn for them.

Updated Indonesian translation

Updated German translation

Updated Hebrew translation

Adwaita: Make selector more specific

A * selector applies to all widgets, so even GtkBox or GtkGrid - and
most importantly GtkListBoxRow - need to recompute their style because
of the * selector.

By using a more specific one, these common cases aren't affected
anymore.

Fixes slowdowns in gtk3-demo's listbox demo and in gnome-software.

combobox: Add a special class for the button in the combobox

That way, the GTK engine doesn't think that the general .button CSS
might potentially apply to it.
And because combobox button is overly complex and stupid, it cannot be
cached.

So buttons thought they cannot ever cache anything because they might
suddenly end up inside a combobox without noticing and then they'd need
to round their corners differently. Of course they're just regular
"Remove" buttons like all the other 100s of "Remove" buttons in
gnome-software. But hey, better not cache anything for them and
recompute their CSS every time the :hover state changes on one of the
rows.

css: Add gtk_css_change_to_string()

Nobody is able to look up those hex values.

css: Fix GtkCssChange enum

Get rid of unused values (POSITION_ENUM) and use different values for
NAME and ID.

cssnode: Loosen requirements for style caching

We can actually share :first-child/:last-child related things now,
because we special case them. So the only positions we cannot cache are
nth-child/nth-last-child.

This should take care of a lot of Adwaita's styling.

Updated Hungarian translation

HighContrast: GtkListBox fixes

Make flat buttons in list-rows work.

Updated Portuguese translation

Adwaita: Theme calendar day names differently

Just to show how its done. This will need some tweaks.

flow box: Avoid a critical in keynav

We were not taking into account that gtk_flow_box_get_next_focusable
can return NULL, in all callers. Fix that.

testflowbox: Add an example with buttons

This example populates a flow box with buttons, and makes the
flow box children unfocusable, with the intention that the focus
moves directly between the buttons. Currently, keynav does not
work at all in this case.

gdkgears: Add a popover

As pointed out in bug 754143, this reveals a crash in the Wayland
backend.

Updated Polish translation

places view row: Use proper gettext macros

P_() is meant only for property nicks and blurbs. Everything
else should use _().

Follow-up to 2d83dd62173fc5e651e7df95f331ad5cc46a29cf

http://bugzilla.gnome.org/show_bug.cgi?id=754086

inspector: Handle object destruction without row references

This way, we can live without row references.

A side effect is that opening the inspector on the gtk-demo list box
example now only takes 0.5s instead of the previous 3 minutes.

inspector: Implement find_object() without GtkTreeRowReference

Tree row references are slow to update. And we update the widget list
quite a bit, so when it grows it can get REALLY slow.

inspector: Refactor object tree construction

Instead of having lots of if statements, create a vtable. This will be
useful for the following patches.

window: Only update the inspector once

Instead of queueing a new idle handler every time we call
gtk_window_update_debugging(), only queue one if none is queued that.

Saves a lot of work, in particular when templates create context menus
for every row in a large listbox as in the gtk-demo listbox example.

listbox: Mark activatable rows with a .activatable style class

Do not use .button anymore.

This is for 2 reasons:
1. The styling is seperate in our themes, so it doesn't make sense to
   share the style class.
2. Due to the shared styling of .buton, listbox rows inherit all the
   special case styles that exist for buttons - such as linked buttons,
   header buttons, entry buttons, spinbutton buttons, etc. This means
   that the code has to check all these special cases all the time and
   for listbox rows, this is very slow.

Updated Hungarian translation

Inspector: Protect against childless windows

Updated Portuguese translation

places view: Use proper gettext macros

P_() is meant only for property nicks and blurbs. Everything
else should use _().

http://bugzilla.gnome.org/show_bug.cgi?id=754086

Updated Chinese (Taiwan) translation

Updated Chinese (Taiwan) translation

flow box: Document incompatible configurations

Explain in the docs that using a model is incompatible with
using filter or sort functions.

flow box: Warn for nonworking configurations

GtkFlowBox sort and filter functionality does not work with
models. Issue a warning in this case.

list box: Document incompatible configurations

Explain in the docs that using a model is incompatible with
using filter or sort functions.

http://bugzilla.gnome.org/show_bug.cgi?id=752615

list box: Warn for nonworking configurations

GtkListBox sort and filter functionality does not work with
models. Issue a warning in this case.

http://bugzilla.gnome.org/show_bug.cgi?id=752615

Fix a startup ordering problem

Defer a11y initialization until we have a display. A11y initialization
causes widget classes to be initalized, which in turn needs some
backend-specific information about modifier masks that can't be
obtained before we have a display.

https://bugzilla.gnome.org/show_bug.cgi?id=736125

Bug 736125 - The default gtk accelerator mod mask does not include all default backend modifiers

Inspector: Make monitor rows unactivatable

Fix grammar in "No networks locations found"

Update Galician translations

Updated Galician translations

Updated Spanish translation

Updated Spanish translation

Updated Portuguese translation

Updated Portuguese translation

gtk-demo: Update listbox example with longer list

Add all 388 tweets of the @GTKtoolkit account. This shows the
performance behavior of the listbox (not good with that many rows) and
allows us to quickly notice when things get worse (or better).

And just so I have a place where I can dump how I generated this file:

First, I got Timm Bäder to download me the json for the twitter feed
into a file gtk.json, then I ran the jq tool on it like this:

jq ".[] | if .retweeted_status then .retweeted_status.user.name + \"|\"
+ .retweeted_status.user.screen_name else .user.name + \"|\" +
.user.screen_name end + \"|\" + .text" gtk.json | cat -n | sed
"s/\\s*\([0-9]*\)\t\"\(.*\)\"/\\1|\\2/" > messages.start

jq ".[] | .created_at" gtk.json | sed "s/\"\(.*\)\"/\1/" | while read
in; do date +%s -d "$in"; done > dates

jq ".[] | \"0|\" + if .retweeted_status then .user.screen_name else \"\"
end + \"|\" + (.favorite_count | tostring) + \"|\" + (.retweet_count |
tostring)" gtk.json | sed "s/\"\(.*\)\"/\\1/" > messages.end

paste -d\| messages.start dates messages.end > messages.txt

This whole machinery of going through 3 intermediate files was only
necessary to onvert the dates from ISO format to unix timestamps,
otherwise this could have been a single line.

gtk-demo: Order tweets newest first in listbox example

Updated Slovak translation

Updated Slovak translation

filechooser: check if we can access the parent folder

If we manually enter an unaccessible path in the entry, e.g
"/root/foo.txt", we should receive an error saying that the
folder is not accessible instead of showing the replace
confirmation dialog.

https://bugzilla.gnome.org/show_bug.cgi?id=753969

Revert "gtkoverlay: Optimize size allocations"

This reverts commit 93171cf4f346c0a5e71f274d012c3cacfe3880ba.

This broke overlay controls in page 3 of widget-factory, gnome-photos,
gnome-documents, etc.

list box: Make focusable headers possible

Previously we were assuming that only list box rows could occur
as focus children of a list box, and would crash if that wasn't
the case. This commit handles this case, and integrates focusable
headers into directional keynav and the focus chain.

The typical case of using separators as headers is not affected
by this change.

https://bugzilla.gnome.org/show_bug.cgi?id=753694

GtkOverlay: Update the description

State that the overlays are placed wrt to the GtkOverlay, not
with respect to the main widget. This makes a difference for
small main widgets which are not configured to fill the entire
GtkOverlay.

Revert "popover: Optimize the transition"

This reverts commit 84bd32a09a25bee24ae7740e30bd75727695d744.

It turned out that this did break the transition in some places.

gtkplacesview: bail out if operation cancelled

When an operation is cancelled it's never safe to access
the object itself or the private struct, since it could be
called (and probably is) during finalize.

In case the operation is cancelled, just bail out to fix
the crashes.

Upload Catalan translator

gtkplacessidebar: order mounts by label

We were not ordering them at all, and therefore they were
jumping around

https://bugzilla.gnome.org/show_bug.cgi?id=753925

gtkplacesview: use g_utf8_collate for sorting

Also rename the variable, since we usually use location
for GFiles variables.

gtkplacesview: add a loading property

So clients of the view can know if the view is busy.

https://bugzilla.gnome.org/show_bug.cgi?id=753786

gtkplacesview: improve networks feedback

Add a spinner when networks are being fetched and make
the network section permanent and show a placeholder with
a message that no networks were found in case there are no
networks. In this way users from previous versions won't be
confused with the fact that no networks are shown.

https://bugzilla.gnome.org/show_bug.cgi?id=753786

gtkplacesview: add networks in network:///

Previously we had a network item in the sidebar, which now
is replaced by the network section on other-locations view.
However we were not exposing the networks in network:///.

Fetch them and add them in the network section of other-locations
view.

https://bugzilla.gnome.org/show_bug.cgi?id=753786

gtkplacesview: style fix

https://bugzilla.gnome.org/show_bug.cgi?id=753786

Updated Slovak translation

wayland: Only bind supported unstable pointer gestures global

As the protocol is still considered unstable (meaning not backward
compatible), we should, as stated in the protocol, only bind the version
advertised is the version we implement.

https://bugzilla.gnome.org/show_bug.cgi?id=753856

overlay: Always align widgets relative to the overlay

... not to the contained widget, which might not occupy the whole
overlay when an align mode is set.

overlay: Use actual size of the overlay when positioning

Previously we were using the natural requested size which is unrelated.

Updated Lithuanian translation

treeview: check for pixel cache before setting property

Code exists in the wild that calls this function after the widget has
been destroyed (and the pixel cache released). Simply check that the
pixel cache exists to preserve the existing state.

Update Catalan translation

immodule: Remove unused parameter

Updated POTFILES.skip

gtk-demo: Add a model button demo

This shows how to hook model buttons up to various action types,
or how to create them manually.

GtkTextView: fix margin/padding scroll behaviour

When you move line by line, only padding is
automaticly shown and you need to use Page key to show margin.

This commit also fix cursor going out of the screen bug.

gdkevent: Hold refs to device/source_device

The extra reference will be held from GdkEventPrivate data, so there's
a common place to all events. Without this, events queued after devices/
capabilities disappear (eg. on TTY switch) might hold invalid pointers.
Windowing level operations on those devices (queries, grabs...) are
expected to fail at that time, but we should hold meaningful data for
the regular event handling paths.

https://bugzilla.gnome.org/show_bug.cgi?id=753185

gtkscrolledwindow: Fold kinetic deceleration handling into scroll_event()

In order to play along with child widgets that use scroll events for anything
else than scrolling, it will be better to do this in the bubble phase, so
the child widget has an opportunity to GDK_EVENT_STOP the event before we
trigger kinetic scrolling.

This of course won't work for widgets that choose to reimplement scroll event
handling themselves, they should be smart at resorting to GtkScrolledWindow's
scroll event handling.

This fixes kinetic scrolling kicking in too pervasively on widgets that eg.
implement zoom on scroll events.

https://bugzilla.gnome.org/show_bug.cgi?id=753495

Updated Hungarian translation

placessidebar: set Other Locations uri

By assigning an URI to Other Locations item, we
can programaticaly select it. Fixes a bug in Nautilus,
where the Other Locations item is unselected imediately
after being clicked.

multipress: Only emit preedit signals if needed

When clearing the compose buffer, don't emit preedit-changed
unconditionally. This triggers a recursion which leads to
a crash.

https://bugzilla.gnome.org/show_bug.cgi?id=753644

print operation: let subclasses use a custom paginate

GtkPrintOperation was emitting paginate only if a signal was
connected, this meant that subclassing and overriding the
paginate vfunc lead to the unexpected result that paginate did
not run.
Instead we always emit the signal and use a custom accumulator:
if there is a signal we just run that and avoid the default
handler, otherwise we run the default handler which can be the
one by the subclass or the default handler that just skips
pagination.

Patch by Yevgen Muntyan, fixes #345345

Updated Spanish translation

Updated Spanish translation

3.17.7

wayland: Use g_get_prgname() to get the xdg_surface.set_app_id string

Prior to this patch, the ID of the GtkApplication was always used for
clients which were GtkApplications. This would only be guaranteed to be
correct for D-Bus activatable programs. As a result, all
non-D-Bus-activatable applications would set the wrong ID making the
shell unable to find the corresponding .desktop file.

This change makes it so that the GDK backend always uses the name
passed to g_set_prgname, or the default value if not explicitly set, as
this more often corresponds to the .desktop file.

This means that in order to make D-Bus activatable applications set the
correct application ID, they must, for now, manually call
g_set_prgname() with their application ID (basename of the .desktop
file).

If g_get_prgname() returns NULL, fallback to gdk_get_program_class()
even though it will most likely never be correct according to the
xdg_surface.set_app_id specification.

https://bugzilla.gnome.org/show_bug.cgi?id=746435

gtkplacesview: allow to cancel connection to server

We were not allowing to cancel the operation at all, and at
most the operation was cancelled only when clicked connect again.

Also due to gvfs bug 753735 we actually weren't cancelling
at all, and therefore creating multiple dialogs.

places-view: show a more verbose entry hint

As pointed out by Allan, this is not very helpful. A
call to action label fits better in this case.

flowbox: Sink the floating reference of model-based children

We don't want to leak references if the widget created to represent the
item in the model does not have a floating reference — which is usually
what happens in bindings, as they automatically sink references when
creating new instances.

See commit 6e03e7e8 for the similar change in GtkListBox.

MSVC Introspection Builds: Remove GCC Requirement

Sorry, the last commit added a generated file instead of the
template.

G-I has been updated to not require a Windows GCC installation
anymore to generate the .gir files, so update the NMake Makefiles
that are used for this purpose.

As a result, it is no longer necessary to define time_t for the .gir
generation as we are on the same compiler throughout the process.

Revert "MSVC Introspection Builds: Remove GCC Requirement"

This reverts commit aea7809f55ac520b9d2d5d9c1eea1345fabbd7a7.

MSVC Introspection Builds: Remove GCC Requirement

G-I has been updated to not require a Windows GCC installation
anymore to generate the .gir files, so update the NMake Makefiles
that are used for this purpose.

As a result, it is no longer necessary to define time_t for the .gir
generation as we are on the same compiler throughout the process.

gtk-demo: Use top/bottom-margin for text views

This lets us replace the hack of inserting tags in the first
and last paragraph to set a above/below-paragraph space.

text view: add padding and margin

Add API to set top and bottom margins, and add the CSS padding
from the theme to the programmatically set margins.

https://bugzilla.gnome.org/show_bug.cgi?id=406159